[test][radio] Add axe tests and WCAG conformance reports by michelengelen · Pull Request #48920 · mui/material-ui

michelengelen · 2026-08-04T18:34:19Z

Adds axe coverage and WCAG 2.2 Level A/AA conformance reports for Radio and RadioGroup.

Radio

Result	Count
✅ Supports	23
⚠️ Partially Supports	2
❌ Does Not Support	0
➖ Not Applicable	30

RadioGroup

Result	Count
✅ Supports	7
⚠️ Partially Supports	0
❌ Does Not Support	0
➖ Not Applicable	30
↗ Inherited from Radio	18

Known gaps: focus-indicator contrast and custom icons below 3:1 (1.4.11), and disableRipple removing the focus indicator (2.4.7). No group-level gaps.

Reports: Radio/accessibility.md · RadioGroup/accessibility.md

Important

How to review this PR

Layer 7 of 14 in a stacked series (#48915 → #48926). Its branch is built on a11y/stack/switch (#48919). GitHub cannot chain PR bases across a fork, so every PR in the series targets master instead.

Review only the last commit — d2b70c8. Everything above it belongs to the layers below, and disappears as those merge.

Original work by @mj12albert. Supersedes #48751, rebased onto current master.

Part of the WCAG conformance effort (#14187).

code-infra-dashboard · 2026-08-04T18:37:05Z

Deploy preview

docs/data/material/components/radio-buttons/ColorRadioButtons.js

Bundle size

Bundle	Parsed size	Gzip size
@mui/material	0B^(0.00%)	0B^(0.00%)
@mui/lab	0B^(0.00%)	0B^(0.00%)
@mui/private-theming	0B^(0.00%)	0B^(0.00%)
@mui/system	0B^(0.00%)	0B^(0.00%)
@mui/utils	0B^(0.00%)	0B^(0.00%)

Details of bundle changes

Check out the code infra dashboard for more information about this PR.

michelengelen · 2026-08-18T09:20:00Z

PR review

Nothing merge-blocking. The inputProps → slotProps.input demo modernization is a welcome side fix, and the browser-gated roving-focus test for RadioGroup is correctly skipIf(isJsdom()). The two RadioGroup tests that were deleted (radiogroup role, shared generated name) reappear unchanged under 1.3.1. One parity gap.

Tests (1)

1. 🟡 Radio 2.5.2 skips the cancellation branch the sibling suites assert

Location: packages/mui-material/src/Radio/Radio.test.js:257

it('2.5.2 Pointer Cancellation: activates on the pointer up-event, not the down-event', ...

Button, Checkbox, Switch, and ToggleButton all press on the control, move off the target, release, and assert that nothing activates. The Radio test only asserts that the down-event alone does not select. The abort path — the substance of 2.5.2 — is untested for Radio.

Failure scenario: a regression that commits the selection on pointerdown-plus-pointerup-anywhere passes this test.

Fix: add the press → move to an outside target → release sequence and assert checked stays false, as in the Checkbox suite.

Verdict

Approve after nits - one missing cancellation branch keeps Radio below the parity of its sibling suites.

🤖 Review generated with Claude Code

Applies mui#48916 (comment): the shared reports table starts empty, and each component PR appends its own row. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_013HgD6GEW3UxNRvC2abbTy9

# Conflicts: # packages/mui-material/src/Radio/Radio.test.js # packages/mui-material/src/accessibility.md # test/regressions/demoMeta.ts

JCQuintas · 2026-09-03T09:44:06Z

+| Component                                   | ✅ Supports | ⚠️ Partially Supports | ❌ Does Not Support | ➖ Not Applicable |
+| :------------------------------------------ | :---------- | :-------------------- | :------------------ | :---------------- |
+| [Button](./Button/accessibility.md)         | 23          | 4                     | 0                   | 28                |
+| [Radio](./Radio/accessibility.md)           | 23          | 2                     | 0                   | 30                |
+| [RadioGroup](./RadioGroup/accessibility.md) | 7           | 0                     | 0                   | 30                |


Should this table have an "inherited" column?

where would it inherit from?

RadioGroup has an inherited part, that it inherits from Radio

I feel like it should be displayed here,

inherited

7 (from Radio)

# Conflicts: # packages/mui-material/src/accessibility.md # test/regressions/demoMeta.test.ts # test/regressions/demoMeta.ts

Reach controls with user.tab instead of programmatic focus in the 2.1.1, 2.1.2, and 2.4.3 tests. Add a 2.4.3 test: a disabled radio is not in the tab order. Add a RadioGroup test: arrow navigation skips a disabled radio. The roving arrow-key test in RadioGroup.test.js already covers focus and value movement. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_013HgD6GEW3UxNRvC2abbTy9

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_013HgD6GEW3UxNRvC2abbTy9

# Conflicts: # packages/mui-material/src/accessibility.md

# Conflicts: # packages/mui-material/src/accessibility.md # test/regressions/demoMeta.test.ts # test/regressions/demoMeta.ts

The merge commit kept conflict markers in demoMeta.ts and its test file. Restore both sides: the progress entries from master and the radio entries from the branch. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_013HgD6GEW3UxNRvC2abbTy9

# Conflicts: # packages/mui-material/src/accessibility.md # test/regressions/demoMeta.test.ts

Co-authored-by: Albert Yu <albert@albertyu.co> Co-authored-by: Claude Fable 5 <noreply@anthropic.com>

This was referenced Aug 4, 2026

[test][toggle button] Add axe tests and WCAG conformance reports #48921

Merged

[test][radio] Add axe+WCAG test coverage and docs #48751

Closed

michelengelen force-pushed the a11y/stack/radio branch from fd73e27 to 586dbce Compare August 5, 2026 09:56

michelengelen added test accessibility a11y scope: radio Changes related to the radio. scope: radio group Changes related to the radio group. labels Aug 5, 2026

michelengelen force-pushed the a11y/stack/radio branch 5 times, most recently from 53cd134 to 8dde96f Compare August 6, 2026 10:43

michelengelen requested review from silviuaavram and siriwatknp August 7, 2026 10:15

michelengelen marked this pull request as ready for review August 7, 2026 10:15

michelengelen force-pushed the a11y/stack/radio branch 3 times, most recently from ed67a1e to d2b70c8 Compare August 11, 2026 12:25

github-actions Bot added the PR: out-of-date The pull request has merge conflicts and can't be merged. label Aug 19, 2026

[test][radio] Add axe tests and WCAG conformance reports

621aa78

michelengelen force-pushed the a11y/stack/radio branch from d2b70c8 to 621aa78 Compare August 26, 2026 10:35

github-actions Bot removed the PR: out-of-date The pull request has merge conflicts and can't be merged. label Aug 26, 2026

Add the Radio and RadioGroup report row

2eda212

Applies mui#48916 (comment): the shared reports table starts empty, and each component PR appends its own row. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_013HgD6GEW3UxNRvC2abbTy9

github-actions Bot added the PR: out-of-date The pull request has merge conflicts and can't be merged. label Aug 27, 2026

Merge branch 'master' into a11y/stack/radio

4d77298

# Conflicts: # packages/mui-material/src/Radio/Radio.test.js # packages/mui-material/src/accessibility.md # test/regressions/demoMeta.ts

github-actions Bot removed PR: out-of-date The pull request has merge conflicts and can't be merged. labels Sep 1, 2026

github-actions Bot added the PR: out-of-date The pull request has merge conflicts and can't be merged. label Sep 2, 2026

JCQuintas approved these changes Sep 3, 2026

View reviewed changes

silviuaavram reviewed Sep 3, 2026

View reviewed changes

michelengelen and others added 2 commits September 4, 2026 09:39

Merge branch 'master' into a11y/stack/radio

eaf1788

# Conflicts: # packages/mui-material/src/accessibility.md # test/regressions/demoMeta.test.ts # test/regressions/demoMeta.ts

github-actions Bot removed the PR: out-of-date The pull request has merge conflicts and can't be merged. label Sep 4, 2026

Format the RadioGroup disabled-radio test

141af43

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_013HgD6GEW3UxNRvC2abbTy9

github-actions Bot added the PR: out-of-date The pull request has merge conflicts and can't be merged. label Sep 4, 2026

Merge branch 'master' into a11y/stack/radio

bfee604

# Conflicts: # packages/mui-material/src/accessibility.md

github-actions Bot added PR: out-of-date The pull request has merge conflicts and can't be merged. and removed PR: out-of-date The pull request has merge conflicts and can't be merged. labels Sep 4, 2026

Merge branch 'master' into a11y/stack/radio

78c2469

# Conflicts: # packages/mui-material/src/accessibility.md # test/regressions/demoMeta.test.ts # test/regressions/demoMeta.ts

github-actions Bot removed the PR: out-of-date The pull request has merge conflicts and can't be merged. label Sep 4, 2026

github-actions Bot added the PR: out-of-date The pull request has merge conflicts and can't be merged. label Sep 4, 2026

Merge branch 'master' into a11y/stack/radio

19730f3

# Conflicts: # packages/mui-material/src/accessibility.md # test/regressions/demoMeta.test.ts

github-actions Bot removed the PR: out-of-date The pull request has merge conflicts and can't be merged. label Sep 7, 2026

Merge branch 'master' into a11y/stack/radio

3fe22f0

michelengelen merged commit e0c9fbc into mui:master Sep 7, 2026
22 checks passed

michelengelen deleted the a11y/stack/radio branch September 7, 2026 13:46

brijeshb42 pushed a commit to brijeshb42/material-ui that referenced this pull request Sep 9, 2026

[test][radio] Add axe tests and WCAG conformance reports (mui#48920)

ac52791

Co-authored-by: Albert Yu <albert@albertyu.co> Co-authored-by: Claude Fable 5 <noreply@anthropic.com>

Navigation Menu

Uh oh!

[test][radio] Add axe tests and WCAG conformance reports - #48920

[test][radio] Add axe tests and WCAG conformance reports#48920
michelengelen merged 11 commits into
mui:masterfrom
michelengelen:a11y/stack/radio

michelengelen commented Aug 4, 2026 •

edited

Loading

Uh oh!

code-infra-dashboard Bot commented Aug 4, 2026 •

edited

Loading

Uh oh!

michelengelen commented Aug 18, 2026

Uh oh!

JCQuintas Sep 3, 2026

Uh oh!

michelengelen Sep 3, 2026

Uh oh!

JCQuintas Sep 3, 2026

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Uh oh!

Conversation

michelengelen commented Aug 4, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

code-infra-dashboard Bot commented Aug 4, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Deploy preview

Bundle size

Uh oh!

michelengelen commented Aug 18, 2026

PR review

Tests (1)

1. 🟡 Radio 2.5.2 skips the cancellation branch the sibling suites assert

Verdict

Uh oh!

JCQuintas Sep 3, 2026

Choose a reason for hiding this comment

Uh oh!

michelengelen Sep 3, 2026

Choose a reason for hiding this comment

Uh oh!

JCQuintas Sep 3, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

michelengelen commented Aug 4, 2026 •

edited

Loading

code-infra-dashboard Bot commented Aug 4, 2026 •

edited

Loading